Pin to glue releases #53
Conversation
set `netnum` to avoid the breaking change. Signed-off-by: Matt Moore <[email protected]>
| name = var.name | ||
| project_id = var.project_id | ||
| regions = var.regions | ||
| netnum_offset = 1 |
There was a problem hiding this comment.
Without this actions are failing with:
module.networking.google_compute_subnetwork.regional["us-central1"]: Destroying... [id=projects/octo-sts/regions/us-central1/subnetworks/octo-sts-us-central1]
Error: Terraform exited with code 1.
│
╵
Error: Process completed with exit code 1.
There was a problem hiding this comment.
Were there any more details in this error? The intent of this change was to be backwards compatible and fall back to the provider project if these values weren't set.
There was a problem hiding this comment.
e.g. netnum_offset should default to 0 if not provided, so I'm curious why this would be causing problems 🤔
There was a problem hiding this comment.
ahhhh i see what's going on. this is the real error:
│ Error: Error when reading or editing Subnetwork: googleapi: Error 400: The subnetwork resource 'projects/octo-sts/regions/us-central1/subnetworks/octo-sts-us-central1' is already being used by 'projects/octo-sts/regions/us-central1/addresses/serverless-ipv4-1706112377164427487', resourceInUseByAnotherResource
Previously the old network module was using a cidr address n+1 than what was given. Now we zero index so we don't waste cidr range, which changed the subnet address being used to start at 0, but the old subnet couldn't be removed because the existing cloud run resources were already using it.
So this change is just matching the previous n+1 cidr behavior as before.
| name = var.name | ||
| project_id = var.project_id | ||
| regions = var.regions | ||
| netnum_offset = 1 |
There was a problem hiding this comment.
Were there any more details in this error? The intent of this change was to be backwards compatible and fall back to the provider project if these values weren't set.
set
netnumto avoid the breaking change.